-
-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Panel scrollbar update #726
Conversation
- Reflow functions now use the *ContentRegion* Width instead of the Width of the Panel - Default Scrollbar Padding is 4 px - to handle panels inside panels which use FILL to be sized correctly some scrollbar properties have been moved up from (Flow)Panel to Container - Content Regions only resize to their VISIBILE childs size instead of accounting for any which resulted in panels to not grow back to smaller sizes when hiding bigger controls - Scrollbar now returns the _barBounds.Width instead of the _textureBar.Width
@@ -146,6 +196,7 @@ public abstract class Container : Control, IEnumerable<Control> { | |||
|
|||
public bool AddChild(Control child) { | |||
if (_children.Contains(child)) return true; | |||
if (_panelScrollbar != null && child == _panelScrollbar) return true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be shortened to
return _panelScrollbar != null && child == _panelScrollbar
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't that return always and not continue to add the child?
Can you please include a description of all of the changes or provide a link to a full summary of where you've described it in Discord? |
Container.csProperties:
Methods:
Scrollbar.csProperties:
Panel.csMethods:
UpdateScrollbar
FlowPanel.csMethods:
|
moved to #749 for a cleaner PR |
No description provided.